Skip to content

Conversation

@mfori
Copy link
Member

@mfori mfori commented May 2, 2025

@mfori mfori added the t-console Issues with this label are in the ownership of the console team. label May 2, 2025
@mfori mfori self-assigned this May 2, 2025
@github-actions github-actions bot added this to the 113th sprint - Console team milestone May 2, 2025
@github-actions github-actions bot added the tested Temporary label used only programatically for some analytics. label May 16, 2025
@mfori mfori requested review from barjin and Copilot May 16, 2025 09:16
@mfori mfori marked this pull request as ready for review May 16, 2025 09:16
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for filtering key-value store keys by adding new query parameters ("collection" and "prefix"). The changes include:

  • Updating the test case to pass the new query parameters.
  • Adding validation for the new parameters in the client and updating the TypeScript interface.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
test/key_value_stores.test.js Updated test query to include new "collection" and "prefix" options
src/resource_clients/key_value_store.ts Added validation and interface properties for "collection" and "prefix" parameters

Copy link
Member

@barjin barjin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty straightforward to me, thanks! 🚀

@barjin
Copy link
Member

barjin commented May 16, 2025

Also, I suppose we should implement this in MemoryStorage in Crawlee too (at least the prefix param) for parity with Platform-based KVS.

Issue in Crawlee here

@mfori mfori merged commit be25137 into master May 16, 2025
7 checks passed
@mfori mfori deleted the feat/load-keys-by-collection branch May 16, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-console Issues with this label are in the ownership of the console team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants